Skip to content

[SYCL][Graph] Add E2E test for recording handlerless queue submissions #20420

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: sycl
Choose a base branch
from
Open

[SYCL][Graph] Add E2E test for recording handlerless queue submissions #20420

wants to merge 6 commits into from
+122 −0

Conversation

mmichel11
Copy link
Contributor

  • Follow-up test case to the bug fixed in [SYCL] Update the kernel function pointer on function copy/move #20389 which was not caught by SYCL graph tests.
  • Exercises graph recording over queue submissions with handlerless APIs. Queue submissions are recorded over a non-inlined function call to provide coverage over now resolved kernel argument capture issues.

@mmichel11 mmichel11 requested a review from a team as a code owner October 21, 2025 15:57
@mmichel11 mmichel11 requested a review from reble October 21, 2025 15:57
@mmichel11 mmichel11 requested a review from slawekptak October 21, 2025 15:58
@mmichel11 mmichel11 marked this pull request as draft October 21, 2025 19:33
@mmichel11 mmichel11 marked this pull request as ready for review October 22, 2025 02:24
@mmichel11
Copy link
Contributor Author

Removed the handlerless ops not supported with command buffers in the OpenCL adapter (prefetch, memadvise), so we can still test this configuration.

reble
reble reviewed Oct 22, 2025
View reviewed changes
sycl/test-e2e/Graph/RecordReplay/handlerless_enqueue_functions.cpp
Comment on lines +12 to +17
#include "../graph_common.hpp"
#include <cstdint>
#include <cstring>
#include <sycl/ext/oneapi/experimental/enqueue_functions.hpp>
#include <sycl/properties/all_properties.hpp>
#include <vector>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks quite verbose to other related tests. Can you please check if includes are actually required, if so we might want to update graph_common.hpp ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@reble reble reble left review comments

@slawekptak slawekptak slawekptak approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mmichel11 @reble @slawekptak